Skip to content

fix: display rewritten query as the formatted query#152

Merged
veksen merged 1 commit into
mainfrom
feat-formatted-query-rewrite
Jun 18, 2026
Merged

fix: display rewritten query as the formatted query#152
veksen merged 1 commit into
mainfrom
feat-formatted-query-rewrite

Conversation

@Xetera

@Xetera Xetera commented Jun 18, 2026

Copy link
Copy Markdown
Collaborator

Should fix the problem with queries showing LIMIT $1 when they're really LIMIT 50

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Query Doctor Analysis

3 queries analyzed

0 regressed · 0 improved · 0 new · 0 removed

2 pre-existing issues

Using assumed statistics (10000000 rows/table). For better results, sync production stats.

More detail → get_ci_run({ runId: "019edb30-7ca0-7714-b224-a4a9a0e6e29d" }) · view run · docs

@veksen

veksen commented Jun 18, 2026

Copy link
Copy Markdown
Member

But now we will get the sqlcommenter back...

there is a temporary hotfix in the front-end to strip it (because some sources wouldn't properly strip)

I'm ok to merge but this should be handled differently in the future

@veksen veksen merged commit 0687237 into main Jun 18, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants